Add a useApi hook for accessing the current api client without having a reference to it #217

airhorns · 2023-06-26T02:21:53Z

We need this for the auth helpers that ideally don't need to be passed an instance of the api every time, like:

export const useSession = () => {
  const api = useApi();
  const [{ data, fetching, error }] = useGet(api.currentSession, {
    select: {
      id: true,
      state: true,
      createdAt: true,
      userId: true
    }
  });
  return data;
}

With this hook, we can export that kind of framework code, and rely on the user <Provider/>-ing us the api client we need

… a reference to it

jasong689

❤️

Add a useApi hook for accessing the current api client without having…

63e1905

… a reference to it

airhorns marked this pull request as ready for review June 26, 2023 02:22

airhorns requested a review from jasong689 June 26, 2023 02:22

jasong689 approved these changes Jun 26, 2023

View reviewed changes

airhorns merged commit 7a542eb into main Jun 26, 2023
3 checks passed

airhorns deleted the use-api-hook branch June 26, 2023 15:45

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add a useApi hook for accessing the current api client without having a reference to it #217

Add a useApi hook for accessing the current api client without having a reference to it #217

airhorns commented Jun 26, 2023 •

edited

Loading

jasong689 left a comment

Add a useApi hook for accessing the current api client without having a reference to it #217

Add a useApi hook for accessing the current api client without having a reference to it #217

Conversation

airhorns commented Jun 26, 2023 • edited Loading

jasong689 left a comment

Choose a reason for hiding this comment

airhorns commented Jun 26, 2023 •

edited

Loading